Port ARIEC native report preview and polish IO FAT table - #125
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d84175cacb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| .Append('/').Append(ResourceName(command.Font)).Append(' ').Append(Number(command.FontSize)).Append(" Tf ") | ||
| .Append("1 0 0 1 ").Append(Number(command.X)).Append(' ').Append(Number(command.BaselineY)).Append(" Tm ") | ||
| .Append('(').Append(EscapeLiteral(safe)).Append(") Tj ET\n"); |
There was a problem hiding this comment.
Honor text widths when serializing PDF commands
When imported project or IED fields exceed their assigned IoFatReportTextCommand.Width, the FixedDocument preview constrains and ellipsizes them, but this PDF path emits an unbounded Tj operation and never uses command.Width. Long names, metadata, or status reasons can therefore spill into adjacent cards or columns in the exported PDF even though the new native preview looks correct; truncate or clip PDF text to the command width so both renderers remain consistent.
Useful? React with 👍 / 👎.
Purpose
Apply the latest field UX feedback to the workbook-driven IO List FAT workspace:
Truevisually red and inactiveFalsevisually greenFAT table polish
True/1uses an exciting red emphasisFalse/0uses an exciting green emphasisNative report engine parity
The old HTML preview path is removed. The new architecture is adapted directly from the project-owned ARIEC60870 report engine:
PDF export and in-workspace preview now consume the same layout command stream, so page size, pagination, cards, table geometry, headers and footers remain aligned.
The preview adds ARIEC-style controls:
Removed
WebBrowserNavigateToStringScope and safety
.arsasexport remain availableDRAFT / LIVEAutomated validation at head
d84175cacb954eb1ef4423623916d75a756e8ff2Validate IO List Testing run 42 — success
Build ARSAS run 791 — success
Validate SV evidence bundles run 74 — success
No review threads are open.